-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed an error description of func, and deleted redundant codes #10778
Fixed an error description of func, and deleted redundant codes #10778
Conversation
057bac9
to
01cf166
Compare
[test] LGTM, holding to merge post-1.3. |
@miaoyq thanks for the PR! |
01cf166
to
ed14146
Compare
@fabianofranz I have rebased the PR. Could you please trigger a test? Thanks! |
[test] |
ed14146
to
d021dcf
Compare
The test still failed, but it didn't seem to be caused by the PR. |
Flaked on #11094 |
Evaluated for origin test up to d021dcf |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9360/) |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9360/) (Image: devenv-rhel7_5085) |
Evaluated for origin merge up to d021dcf |
if kcmdutil.GetFlagBool(cmd, "parameters") || outputFormat == "describe"
can't be executed.The function will return in earlier if the condition is true.